New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Cli 148 #280

Merged

Widcket merged 41 commits into main from CLI-148

May 14, 2021

+690 −2

Contributor

bright-poku commented May 3, 2021 •

edited

Loading

Description

This Pull request add the following commands:

users create
users search
users show
users delete
users update

Also, connection API was created which is used for getting connection details.

The current API supports database connections.

References

Testing

Describe how this can be tested by reviewers. Be specific about anything not tested and reasons why. If this library has unit and/or integration testing, tests should be added for new functionality and existing tests should complete without errors.

Please include any manual steps for testing end-to-end or functionality not covered by unit/integration tests.

Also include details of the environment this PR was developed in (language/platform/browser version).

This change adds test coverage for new/changed/fixed functionality

Checklist

I have added documentation for new/changed functionality in this PR or in auth0.com/docs
All active GitHub checks for tests, formatting, and security are passing
[X ] The correct base branch is being used, if not master

bright-poku and others added 4 commits

April 30, 2021 14:36


          [CLI-148] initial commit: user show, create, list

4a33c7e


          [CLI-148] refactor: users command

1ea48f6


          [CLI-148] refactor users command: add error return value

b5138ff


          Merge branch 'main' into CLI-148

b14abdf

Widcket reviewed

View reviewed changes

internal/cli/users.go Outdated Show resolved Hide resolved

Widcket reviewed

View reviewed changes

internal/auth0/auth0.go Outdated Show resolved Hide resolved

Widcket reviewed

View reviewed changes

internal/cli/users.go Outdated Show resolved Hide resolved

Widcket reviewed

View reviewed changes

internal/cli/users.go Outdated Show resolved Hide resolved

Widcket reviewed

View reviewed changes

internal/cli/users.go Outdated Show resolved Hide resolved

Widcket reviewed

View reviewed changes

internal/cli/users.go Outdated Show resolved Hide resolved

Widcket reviewed

View reviewed changes

internal/cli/users.go Outdated Show resolved Hide resolved


          Update internal/cli/users.go

0520af1

Co-authored-by: Rita Zerrizuela <[email protected]>

Widcket reviewed

View reviewed changes

internal/cli/users.go Show resolved Hide resolved

woloski commented May 3, 2021

make sure passwords are masked when creating users

Widcket reviewed

View reviewed changes

internal/cli/users.go Outdated Show resolved Hide resolved

Widcket reviewed

View reviewed changes

internal/cli/users.go Outdated Show resolved Hide resolved

Widcket reviewed

View reviewed changes

internal/cli/users.go Show resolved Hide resolved

Widcket reviewed

View reviewed changes

internal/cli/users.go Outdated Show resolved Hide resolved

Widcket reviewed

View reviewed changes

internal/cli/users.go Outdated Show resolved Hide resolved

Widcket reviewed

View reviewed changes

internal/cli/users.go Outdated Show resolved Hide resolved

Widcket reviewed

View reviewed changes

internal/cli/users.go Outdated Show resolved Hide resolved

Widcket reviewed

View reviewed changes

internal/cli/users.go Outdated Show resolved Hide resolved

Widcket reviewed

View reviewed changes

internal/auth0/connection.go Outdated Show resolved Hide resolved

Widcket reviewed

View reviewed changes

internal/cli/users.go Show resolved Hide resolved

Widcket reviewed

View reviewed changes

internal/cli/users.go Outdated Show resolved Hide resolved

Widcket reviewed

View reviewed changes

internal/cli/users.go Outdated Show resolved Hide resolved

Widcket reviewed

View reviewed changes

internal/cli/users.go Outdated Show resolved Hide resolved

Widcket reviewed

View reviewed changes

internal/cli/users.go Outdated Show resolved Hide resolved

Widcket reviewed

View reviewed changes

internal/cli/users.go Outdated Show resolved Hide resolved

Widcket reviewed

View reviewed changes

internal/cli/users.go Outdated Show resolved Hide resolved

Contributor

Widcket commented May 7, 2021

BTW, everything else is 👍🏼. Thank you for all the hard work!

Contributor Author

bright-poku commented May 7, 2021

@bright-poku testing this, if I pass the password as a flag, it ignores the value and prompts anyway:

I'd suggest you add some methods to Flag to encapsulate prompting for a password when no value was passed. See https://github.com/auth0/auth0-cli/blob/main/internal/cli/flags.go#L40 as a starting point, and https://github.com/auth0/auth0-cli/blob/main/internal/cli/input.go#L20. Note you'd need a variant for the update method as well (e.g. AskU).

good catch I will look into it. Thanks for the feedback

bright-poku and others added 20 commits

May 10, 2021 07:43


          [CLI-148] refactor: password input

ef74b98


          Merge branch 'CLI-148' of github.com:auth0/auth0-cli into CLI-148

9f51ca0


          [CLI-148] refactor: cleanup spaces

7c21f4d


          [CLI-148] refactor: sort imports

f7c25a8


          Merge branch 'main' into CLI-148

08a3df0


          Merge branch 'main' into CLI-148

c8ffee6


          [CLI-148] refactor: sync with master

d74e2d8


          Merge branch 'main' into CLI-148

e849df5


          [CLI-148] refactor: updated set of scopes

4e8007c


          [CLI-148] refactor: updated order of scopes

4aac3b5


          [CLI-148] refactor: updated scopes in auth file

f44f444


          [CLi-148] refactor: clean up comments

aa381aa


          Merge branch 'main' into CLI-148

73c2c8b


          [CLi-148] refactor: updated scopes to match main branch

e2e649e


          Merge branch 'main' into CLI-148

b83b5c8


          [CLi-148] refactor: cleanup

cd1e2b2


          workflow test

0555fcb


          workflow add back integration

a2c707c


          workflow comment out integration

3a8db6e


          [CLI-148] updated scopes in config generator

811f879

Widcket reviewed

View reviewed changes

internal/cli/users.go Outdated Show resolved Hide resolved

Widcket added 2 commits

May 13, 2021 22:39


          Update internal/cli/users.go

f59c51a


          Fix 400 error

377386c

Widcket approved these changes

View reviewed changes

Contributor

Widcket left a comment

Awesome, thank you for all the hard work!

Widcket merged commit 83f9ea5 into main

Widcket deleted the CLI-148 branch

May 14, 2021 01:57

Widcket linked an issue

that may be closed by this pull request

Add more user management capabilities in the CLI #279

Closed

Widcket mentioned this pull request

Add more user management capabilities in the CLI #279

Closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet